Complete missing Arabic (ar) translations#70383
Open
shahar1 wants to merge 1 commit into
Open
Conversation
The Arabic locale had drifted to 90% coverage as new UI features landed, so Arabic-speaking users saw untranslated English fallbacks across the dashboard, keyboard-shortcuts help, preset filters and the new state stores. The asset and task state stores were previously called "asset store" and "task store". Their Arabic wording is carried over to the renamed keys so the existing translation is not lost, and the dead pre-rename keys — which only ever existed in this locale — are dropped.
Contributor
Author
|
cc: @ahmadtfarhan |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Brings the Arabic (
ar) UI locale back to 100% coverage — it had drifted to 90% (119 missing keys) as new UI features landed, so Arabic-speaking users saw English fallbacks in the dashboard deadlines/alerts panels, the keyboard-shortcuts help, preset filters, run-state counts, and the asset/task state stores.Terminology follows the glossary in
.claude/skills/airflow-translations/locales/ar.mdand the wording already used inar/*.json, including the full six-form CLDR plural set (_zero/_one/_two/_few/_many/_other) for the new pluralisedshowMoreAlertskey, and Western Arabic numerals throughout.breeze ui check-translation-completeness --language arnow reports 100% with 0 missing and 0 TODOs.Recovered wording from the state-store rename
assetStore→assetStateStoreandtaskStore→taskStateStorewere renamed upstream, which orphaned ~30 already-translated Arabic keys and left the new ones untranslated. Rather than re-translate from scratch, the existing Arabic wording is carried over to the new keys (with "المخزن" qualified as "مخزن الحالة" to match the added "State"), and the dead pre-rename keys — which existed only in this locale — are removed.Was generative AI tooling used to co-author this PR?
Generated-by: Claude Code (Opus 4.8) following the guidelines